Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Notes update #1380

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Notes update #1380

wants to merge 4 commits into from

Conversation

nil-malh
Copy link
Contributor

@nil-malh nil-malh commented Jun 8, 2023

note.py

Fixed the naming of the get_my_notes it was unclear the method returns all the notes from all the user's friends

Created two new methods get_note_by_user that requires the username it returns a NoteResponse object and get_note_content_by_user that requires the username and returns the NoteRequest object, that only contains the text and uuid field.

Fixed the get_all_notes that was returning a json text now it will return an array of NoteResponse that ease the process of retrieving information about a note like the username

extractor.py

Added an extract_note that returns a NoteResponse and extract_note_content method that returns a NoteRequest

types.py

Fixed some BaseModel declarations that were not named according to the response content

Updated the wiki as well

nil-malh added 4 commits June 8, 2023 14:26
…s all the notes from all the user's friends

Created two new methods get_note_by_user that requires the username it returns a NoteResponse object and get_note_content_by_user that requires the username and returns the NoteRequest object, that only contains the text and uuid field.

Fixed the get_all_notes that was returning a json text now it will return an array of NoteResponse
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant